Skip to content

Add 'info' to set of commands #28

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
adamrtalbot wants to merge 2 commits into
base: dev
Choose a base branch
from
Open

Add 'info' to set of commands #28

adamrtalbot wants to merge 2 commits into from

Conversation

@adamrtalbot
Copy link
Collaborator

@adamrtalbot adamrtalbot commented Jun 23, 2023

Additions:

  • info added to supported commands
  • Add example yml file in tests/

Changes:

  • the stdout is logged to console to see current status.

@adamrtalbot
Copy link
Collaborator Author

adamrtalbot commented Jun 23, 2023

Fixes #27

@adamrtalbot
Add 'info' to set of commands
a307467 
Additions:
 - info added to supported commands
 - Add example yml file in tests/

Changes:
 - the stdout is logged to console to see current status.
@adamrtalbot adamrtalbot requested a review from ejseqera June 27, 2023 13:04
@ejseqera ejseqera changed the base branch from main to dev June 28, 2023 17:59
@ejseqera
Copy link
Owner

ejseqera commented Jun 28, 2023

Thanks @adamrtalbot! I'm wondering if we want to run tw info by default before we parse the e2e yaml? Then we don't have to an empty info: key to the yaml. What do you think?

@adamrtalbot
Copy link
Collaborator Author

adamrtalbot commented Jul 17, 2023

I was thinking it would be explicit, i.e. only run info when specified in the YAML. We could use info: true?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ejseqera ejseqera Awaiting requested review from ejseqera

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@adamrtalbot @ejseqera